Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

feat: migrate heatmap, horizon, iframe and markup #367

Merged
merged 2 commits into from
Apr 15, 2020

Conversation

kristw
Copy link
Contributor

@kristw kristw commented Apr 15, 2020

🏆 Enhancements

  • Migrate heatmap, horizon, iframe and markup plugin

@kristw kristw requested a review from a team as a code owner April 15, 2020 21:14
@vercel
Copy link

vercel bot commented Apr 15, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/superset/superset-ui/835l2kbpg
✅ Preview: https://superset-ui-git-kristw-migrate-force.superset.now.sh

@codecov
Copy link

codecov bot commented Apr 15, 2020

Codecov Report

Merging #367 into master will decrease coverage by 3.97%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #367      +/-   ##
==========================================
- Coverage   51.20%   47.22%   -3.98%     
==========================================
  Files         126      139      +13     
  Lines        2695     2922     +227     
  Branches      394      403       +9     
==========================================
  Hits         1380     1380              
- Misses       1286     1513     +227     
  Partials       29       29              
Impacted Files Coverage Δ
plugins/legacy-plugin-chart-heatmap/src/Heatmap.js 0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-heatmap/src/index.js 0.00% <0.00%> (ø)
.../legacy-plugin-chart-heatmap/src/transformProps.js 0.00% <0.00%> (ø)
...s/legacy-plugin-chart-horizon/src/HorizonChart.jsx 0.00% <0.00%> (ø)
...ins/legacy-plugin-chart-horizon/src/HorizonRow.jsx 0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-horizon/src/index.js 0.00% <0.00%> (ø)
.../legacy-plugin-chart-horizon/src/transformProps.js 0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-iframe/src/Iframe.jsx 0.00% <0.00%> (ø)
plugins/legacy-plugin-chart-iframe/src/index.js 0.00% <0.00%> (ø)
...s/legacy-plugin-chart-iframe/src/transformProps.js 0.00% <0.00%> (ø)
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 031d91d...3f7c1a8. Read the comment docs.

@kristw kristw changed the title feat: migrate heatmap feat: migrate heatmap, horizon, iframe and markup Apr 15, 2020
test: /\.jsx?$/,
use: config.module.rules[0].use,
});

config.module.rules.push({
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This rule only applies to typescript. but there are older packages that are jsx.

.register();
```

Then use it via `SuperChart`. See [storybook](https://apache-superset.github.io/superset-ui-plugins/?selectedKind=plugin-chart-horizon) for more details.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These storybook links would also need an update. But we can probably do it in a batch later.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. I'll focus on making them compile and storybook work first.

Copy link
Contributor

@ktmud ktmud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@kristw kristw merged commit 152e457 into master Apr 15, 2020
@delete-merged-branch delete-merged-branch bot deleted the kristw--migrate-force branch April 15, 2020 22:44
kristw pushed a commit that referenced this pull request Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants